Skip to content

Better /role and /timeout#30

Closed
MCWinBuild wants to merge 8 commits intoDeclanChidlow:mainfrom
MCWinBuild:main
Closed

Better /role and /timeout#30
MCWinBuild wants to merge 8 commits intoDeclanChidlow:mainfrom
MCWinBuild:main

Conversation

@MCWinBuild
Copy link
Copy Markdown

@MCWinBuild MCWinBuild commented Mar 27, 2026 •
edited
Loading

There may be some problems that you get error in /bot bun run build, and you may need to delete /dist, bun.lock , and other cache. Sometime I get this, but sometime not.
However, I can run it successfully.

Oh Yes, it seems reply and timeout still have bugs now like /warn before. I didn't fix it.

Support to log Timeout with /logs modaction <#channel>
Support to Timeout ≥2 Users at once.
Support to log Timeout with /warns
Support to /role add ≥2 Users at once.

If you find any bugs, welcome to tell me,

@DeclanChidlow
Copy link
Copy Markdown
Owner

DeclanChidlow commented Apr 1, 2026

Thanks so much for the PR. It seems functional, but for purpose of maintenance and code quality:

  1. Please scope this PR to only cover changes to timeout functionality. Role functionality should be covered in a separate PR.

  2. There are a lot of formatting changes and rearranged sections of code that make it difficult for me to see exactly what changes have been made. Could you please avoid out-of-scope changes, and bring formatting inline with these guidelines? I've added my Prettier config to the repository's root directory as well. I fear I must have accidentally removed it while restructuring the bot a while back.

@DeclanChidlow DeclanChidlow added the enhancement New feature or request label Apr 1, 2026
@MCWinBuild
Copy link
Copy Markdown
Author

MCWinBuild commented Apr 1, 2026 •
edited
Loading

I'll re-submit the PR once I have a moment. Apologies for the delay.

@MCWinBuild MCWinBuild closed this Apr 1, 2026
@DeclanChidlow
Copy link
Copy Markdown
Owner

DeclanChidlow commented Apr 1, 2026

Don't appologise! Completely on me for not having stricter contributing guidelines. I've been working on some for this very purpose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@MCWinBuild @DeclanChidlow